• Switch back to type-safe action bindings #2318 👈 (View in Graphite)
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

No actionable comments were generated in the recent review. 🎉

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Greptile Summary

This PR reverts the _Untyped binding workaround introduced in #2299 (commit 2db4303) — where CodecovAction_Untyped and CodeqlAction*_Untyped were used due to a github-workflows-kt API breaking change — and restores the original type-safe action bindings now that the library issue has been resolved.

Key changes:

• branches-and-prs.main.kts & release.main.kts: Replace CodecovAction_Untyped(failCiIfError_Untyped = "true") with the properly-typed CodecovAction(failCiIfError = true) — a Boolean instead of a String.
• codeql-analysis.main.kts: Replace CodeqlActionInit_Untyped and CodeqlActionAnalyze_Untyped with their typed counterparts, and relax the binding version range from [v4.32.4,v5-alpha) back to [v4,v5-alpha) to reflect that typed bindings are now available for the full v4 release line.
• The generated .yaml files are unchanged — both typed and untyped bindings for these actions produce identical YAML output, which is also confirmed by the existing check_yaml_consistency CI job that would catch any divergence.

Confidence Score: 5/5

• This PR is safe to merge — it is a pure API-surface refactoring with no functional or generated-output changes.
• All three changes are mechanical substitutions of _Untyped binding classes with their typed equivalents. The generated YAML files are unaffected (confirmed by the empty diff against the .yaml files), and the existing check_yaml_consistency CI job would catch any divergence. The version-range widening in codeql-analysis.main.kts is intentional and expected — typed bindings are now available for the full v4 range, not just v4.32.4+.
• No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Push / PR / MergeGroup] --> B{Workflow Trigger}

    B --> W1[branches-and-prs.yaml]
    B --> W2[codeql-analysis.yaml]
    B --> W3[release.yaml]

    W1 --> C1[check_yaml_consistency]
    C1 --> C2[check_all_workflow_yaml_consistency]
    C2 --> C3[build-and-verify matrix]
    C3 --> C4["CodecovAction\nfailCiIfError = true"]

    W2 --> D1[check_yaml_consistency]
    D1 --> D2[codeql-build matrix]
    D2 --> D3["CodeqlActionInit\n(typed)"]
    D3 --> D4[Build Spock Classes]
    D4 --> D5["CodeqlActionAnalyze\n(typed)"]

    W3 --> E1[check_yaml_consistency]
    E1 --> E2[build-and-verify matrix]
    E2 --> E3["CodecovAction\nfailCiIfError = true"]
    E3 --> E4[release-spock]
    E4 --> E5[publish-release-docs]

    style C4 fill:#90EE90
    style D3 fill:#90EE90
    style D5 fill:#90EE90
    style E3 fill:#90EE90

_{Last reviewed commit: 034a49d}

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.16%. Comparing base (8d380cc) to head (b7b6d3f).

@@            Coverage Diff            @@
##             master    #2318   +/-   ##
=========================================
  Coverage     82.16%   82.16%           
  Complexity     4822     4822           
=========================================
  Files           472      472           
  Lines         15036    15036           
  Branches       1905     1905           
=========================================
+ Hits          12354    12355    +1     
  Misses         1989     1989           
+ Partials        693      692    -1